-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reference interpolation and integration #3
Conversation
I also decided to flatten the package structure a bit. Up for discussion. |
c3e28a8
to
8f32174
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #3 +/- ##
============================================
- Coverage 100.00% 54.77% -45.23%
============================================
Files 2 5 +3
Lines 17 199 +182
============================================
+ Hits 17 109 +92
- Misses 0 90 +90
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a few comments.
@maltezfaria Would you like me to bring in Vioreanu-Rokhlin quadratures (for now, interpolation later perhaps) for tris/tets into this branch?
I think we should, but I suggest we move forward with the domain/mesh part, then with integral operators, so that we get the ball rolling. But yes, that is definitely something we should keep on the radar for the near future. Maybe open an issue? |
Yes, looks good to me. |
Implement the basic functionality needed to integrate and interpolate over an
AbstractReferenceShape
.Things we may want to cover: